Skip to content

chore(bestpractices): mirror passing-tier reference + CHANGELOG.md#20

Merged
aksOps merged 2 commits intomainfrom
chore/bestpractices-passing-alignment
May 1, 2026
Merged

chore(bestpractices): mirror passing-tier reference + CHANGELOG.md#20
aksOps merged 2 commits intomainfrom
chore/bestpractices-passing-alignment

Conversation

@aksOps
Copy link
Copy Markdown
Contributor

@aksOps aksOps commented May 1, 2026

Summary

  • Mirror RandomCodeSpace/docsiq's working .bestpractices.json shape (project 12628 — passing) so BadgeApp ingestion applies our intent unambiguously
  • Add CHANGELOG.md (Keep-a-Changelog 1.1.0) and point release_notes_justification at it; resolves BadgeApp's autofill heuristic for release_notes
  • Drop documentation_basics_* keys from JSON and the matching lint entry — docsiq passes without it, and BadgeApp's autofill keeps overriding our local Met value with Unmet
  • Reformat contribution_requirements_justification to prose+URL combined (matches docsiq's persisted format)
  • Add 7 metadata fields ($schema, project_id, level: "passing", badge_url, project_page_url, evidence, audit); update the lint workflow's meta allowlist

Final: 66 criteria answered (63 Met / 0 Unmet / 3 N/A).

After merge, the maintainer clicks Save (and continue) 🤖 once on https://www.bestpractices.dev/projects/12716/edit to re-ingest. The two criteria where BadgeApp persisted stale text (contribution_requirements, vulnerability_report_process) may need a one-time manual save on the edit page.

Test plan

  • Best Practices JSON Lint workflow passes on this PR
  • CI workflow passes (no Go / UI source changes)
  • After merge: maintainer clicks "Save (and continue) 🤖" on the BadgeApp edit page; passing badge flips from in_progress to passing
  • Verify https://www.bestpractices.dev/projects/12716/badge renders the green "passing" badge

🤖 Generated with Claude Code

aksOps and others added 2 commits May 1, 2026 11:29
…LOG.md

Mirror docsiq's working .bestpractices.json (project 12628 — passing)
so the BadgeApp at bestpractices.dev applies our intent unambiguously:

- Add 7 metadata fields (\$schema, project_id, level: "passing",
  badge_url, project_page_url, evidence, audit). Update the lint
  workflow's meta allowlist to accept them.
- Add CHANGELOG.md (Keep-a-Changelog 1.1.0) and point
  release_notes_justification at it. With CHANGELOG.md present,
  BadgeApp's autofill heuristic resolves release_notes to Met
  instead of overriding to Unmet.
- Reformat contribution_requirements_justification to prose+URL
  combined (matches docsiq's persisted format).
- Drop documentation_basics_* keys + the matching lint entry; docsiq
  passes without it and BadgeApp's autofill keeps overriding our
  local Met value with Unmet ("No appropriate folder found").

Final: 66 criteria answered (63 Met / 0 Unmet / 3 N/A).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 1, 2026

@aksOps aksOps merged commit 261279f into main May 1, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant